Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary of changes
I noticed while working on #326 that there was a weird setup going on with the MAX32625 targets, where there were two CMake targets for the "boot" (with bootloader) and "no boot" (no bootloader, uses full memory area) versions. Their linker scripts are identical except different values for MBED_APP_START and MBED_APP_SIZE. This is a classic situation where the new memory bank config system can help make things simpler and remove duplicate linker scripts! I removed the boot/no boot stuff and just had one of the targets configure the memory bank size.
Impact of changes
Migration actions required
Documentation
None
Pull request type
Test results
Unfortunately I do not have a board to test with, but I did verify that this change compiles.